-
-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modified behavior of --help/-h and added commands -H/--help--all #1568
Conversation
Modified the behavior of --help and added new command --help--all
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## devel #1568 +/- ##
==========================================
- Coverage 78.39% 78.34% -0.06%
==========================================
Files 56 56
Lines 6558 6569 +11
==========================================
+ Hits 5141 5146 +5
- Misses 1417 1423 +6
☔ View full report in Codecov by Sentry. |
Should I add coverage for the init file? |
wrote the method to avoid having redundant if/else statement when trying to display hidden help messages
Just pushed a commit based on the feedback from the review. |
help_all method is only called once in get_parser method.
changes based on feedback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mendi03 this looks good. Thanks for adding support for this feature!
Closes #1453
--help
/-h
. Some subcommands and there respective help messages were removed since they are only used for development purposes, it will now only show less subcommands.--help-all
/-H
, which will now do what--help
did before. It will show all subcommands and their help message.Output of
-h
and--help
:Output of
-H
and--help-all